Skip to content

[Win32] Properly reinitialize image of items on DPI changes#3074

Open
HeikoKlare wants to merge 1 commit intoeclipse-platform:masterfrom
vi-eclipse:issue-3073
Open

[Win32] Properly reinitialize image of items on DPI changes#3074
HeikoKlare wants to merge 1 commit intoeclipse-platform:masterfrom
vi-eclipse:issue-3073

Conversation

@HeikoKlare
Copy link
Contributor

The current logic for reinitializing the image of an item on DPI changes is insufficient as the default implementation of the called setImage() method and many of the overwrites of that method do an early return in case the image equals to the one that has already been set.

In order to force a reinitialization of the image, this change ensures that the image is first removed and then reset.

Fixes #3073

The current logic for reinitializing the image of an item on DPI changes
is insufficient as the default implementation of the called setImage()
method and many of the overwrites of that method do an early return in
case the image equals to the one that has already been set.

In order to force a reinitialization of the image, this change ensures
that the image is first removed and then reset.

Fixes eclipse-platform#3073
@github-actions
Copy link
Contributor

Test Results

  176 files  ±0    176 suites  ±0   28m 32s ⏱️ + 1m 3s
4 693 tests ±0  4 671 ✅ ±0  22 💤 ±0  0 ❌ ±0 
  489 runs  ±0    483 ✅ ±0   6 💤 ±0  0 ❌ ±0 

Results for commit dff793a. ± Comparison against base commit 9c79cf6.

@HeikoKlare HeikoKlare marked this pull request as ready for review February 11, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Toolbar items wrongly scaled after DPI change Tool Bar Item Disappears after DPI Change and Enabled State Change

1 participant